Skip to content

Turning in the word frequency toolbox#42

Open
linnealaux wants to merge 1 commit intosd16spring:masterfrom
linnealaux:master
Open

Turning in the word frequency toolbox#42
linnealaux wants to merge 1 commit intosd16spring:masterfrom
linnealaux:master

Conversation

@linnealaux
Copy link

This change is Review on Reviewable

@LucyWilcox
Copy link

Review status: 0 of 1 files reviewed at latest revision, 3 unresolved discussions.


frequency.py, line 32 [r1] (raw file):
This could be done more concisely.


frequency.py, line 46 [r1] (raw file):
I'm not sure I understand what you're doing here, why not just say d[c]=1?


frequency.py, line 74 [r1] (raw file):
Could stand to be in an if name equals main statement.


Comments from the review on Reviewable.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants